Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

js/formidable.js (1)

1147-1155: ⚠️ Potential issue | 🟠 Major

Custom <div> errors still miss id/role, so aria-describedby can point to a non-existent node.

Line 1153 only adds frm_error on the parsed node. In this branch, the generated element still doesn’t receive the id from addFieldError (and ignores alert role), which breaks the accessibility linkage/cleanup path.

🐛 Proposed fix

 function getErrorHtml( errorMessage, id ) {
 	const roleString = frm_js.include_alert_role ? 'role="alert"' : '';

 	if ( ! errorMessage.startsWith( '<div' ) ) {
 		return `<div class="frm_error" ${ roleString } id="${ id }">${ errorMessage }</div>`;
 	}

 	const tempDiv = document.createElement( 'div' );
 	tempDiv.innerHTML = errorMessage;
-	tempDiv.firstChild.classList.add( 'frm_error' );
+	const existingWrapper = tempDiv.firstElementChild;
+	if ( existingWrapper ) {
+		existingWrapper.classList.add( 'frm_error' );
+		existingWrapper.id = id;
+		if ( frm_js.include_alert_role ) {
+			existingWrapper.setAttribute( 'role', 'alert' );
+		}
+	}

 	return tempDiv.innerHTML;
 }

#!/bin/bash
# Verify current helper behavior around parsed custom HTML error markup.
sed -n '1140,1195p' js/formidable.js

# Confirm parsed-div branch currently mutates firstChild and does not set id/role.
rg -n "getErrorHtml|startsWith\\( '<div' \\)|firstChild\\.classList|existingWrapper|setAttribute\\( 'role'|\\.id\\s*=\\s*id" js/formidable.js

# Verify aria-describedby and cleanup depend on error element id.
rg -n "aria-describedby|getErrorElementId|errorMessage\\.id|removeFieldError" js/formidable.js

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@js/formidable.js` around lines 1147 - 1155, The parsed-HTML branch fails to
apply the id and role attributes that addFieldError expects, breaking
aria-describedby and cleanup; in the function that builds error markup (the
branch that creates tempDiv and uses tempDiv.firstChild), after adding the
frm_error class to tempDiv.firstChild also set its id to the same id value and
apply the roleString (or setAttribute('role', ...)) so the parsed node gets the
same id/role as the non-parsed branch; update getErrorHtml/addFieldError related
logic to ensure tempDiv.firstChild receives id and roleString before returning
tempDiv.innerHTML.